Code Review

Found 4 issues (filtered from 17 raw findings, threshold: score >= 80):

1. JSDoc claims 15 stages, actual array has 17 (stale comment after refactor)
   The JSDoc reads "Reduced from 19 stages (6+6 extract/write) to 15 stages (5+5 extract/write)" but REQUIRED_STAGE_SEQUENCE actually contains 17 entries (1 + 5 + 5 + 4 + 1 + 1).
   

   BetterPrompt/packages/plugin/mcp/tools/get-run-progress.ts

   Lines 32 to 34 in 07e6d21

   	/**
   	* 5-dimension pipeline stage sequence (v2).
   	* Reduced from 19 stages (6+6 extract/write) to 15 stages (5+5 extract/write).

2. Inline comment references "19-entry array" but sequence is now 17 entries (stale count)
   

   BetterPrompt/packages/plugin/mcp/tools/get-run-progress.ts

   Line 220 in 07e6d21

   // Only include incomplete/failed stages instead of the full 19-entry array.

3. New MCP tools missing from docs/agent/PLUGIN.md (CLAUDE.md: "Keep Agent Docs Updated")
   get-user-prefs.ts, save-user-prefs.ts, and project-filters.ts are new files not listed in the File Map or MCP Tools table.
   https://github.com/onlycastle/BetterPrompt/blob/07e6d215ad1c9f7846a5ec7f95f0ea88a23e4770/docs/agent/PLUGIN.md

4. PLUGIN.md stage list may have drifted from code (flagged in PRs feat(plugin): add structured debug logging and skill progress reporting #179 and refactor(plugin): file logging, install diagnostics, and UI polish #180)
   The "Required stages" list and model tiering table are manually maintained. PRs feat(plugin): add structured debug logging and skill progress reporting #179 and refactor(plugin): file logging, install diagnostics, and UI polish #180 both caught this same drift pattern. Verify the doc's stage list matches REQUIRED_STAGE_NAMES in stage-db.ts.
   https://github.com/onlycastle/BetterPrompt/blob/07e6d215ad1c9f7846a5ec7f95f0ea88a23e4770/packages/plugin/lib/stage-db.ts

Also noted (below threshold, informational):

• 7 No Fallback Policy potential violations in utility functions — scored low because they're at system boundaries (file reads, stdin parsing) where files may not exist on first run
• 1 latent hasArtifact null-check bug in get-run-progress.ts:105 — guarded in practice by caller
• 1 test fixture using legacy score keys in save-domain-results.test.ts — tests pass (504/504)

Generated with Claude Code

_{If useful, react with a thumbs-up. Otherwise, thumbs-down.}